Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@bampakoa thanks for the v22 work! 🎉

I dug into the bold/italic icon question. Good news: it's not caused by the v22 upgrade and it's not a library bug — it's a long-standing config issue in the demo app itself (src/app/app.module.ts), which this PR doesn't touch (the file is identical to master).

Root cause: the demo passes a custom buttonsConfig to NgxWigModule.forRoot(...) that overrides the default bold/italic buttons with the wrong icon class names:

bold:   { ..., styleClass: 'bold',   icon: 'icon-bold' },
italic: { ..., styleClass: 'italic', icon: 'icon-italic' },

The library's CSS only defines the nwe--prefixed classes (.nwe-icon-bold, .nwe-icon-italic). There's no .icon-bold / .icon-italic rule, so the background SVG never loads → blank button. The other buttons (UL, OL, link, underline) come straight from the library defaults, which already use the correct nwe-icon-* classes — that's why they render fine.

The class names were renamed icon-* → nwe-icon-* back in 1cc0c67, but this demo override was never updated, so it's been broken on master too — the v22 upgrade just made it more noticeable.

Fix (in src/app/app.module.ts):

-    icon: 'icon-bold'
+    icon: 'nwe-icon-bold'
...
-    icon: 'icon-italic'
+    icon: 'nwe-icon-italic'

I verified locally on this branch by building the lib + running the demo: with that two-line change both the B and I icons render correctly alongside the rest. The styleClass: 'bold'/'italic' values are harmless (no CSS depends on them), so only the icon values need fixing.

Opened #287 against master with the two-line fix for this. Since it's independent of the v22 work, it can merge separately and you'll pick it up on rebase. 👍

@bampakoa https://github.com/stevermeister/ngx-wig/pull/287/changes

Hello @behdi! We plan to release it this week. Stay tuned 🙂

@stevermeister can you have a quick look so that I can merge it to the main branch?